Skip to content

[VL] Add datasource add-split and read metrics#10787

Merged
FelixYBW merged 1 commit intoapache:mainfrom
rui-mo:wip_ds_metric
Sep 23, 2025
Merged

[VL] Add datasource add-split and read metrics#10787
FelixYBW merged 1 commit intoapache:mainfrom
rui-mo:wip_ds_metric

Conversation

@rui-mo
Copy link
Contributor

@rui-mo rui-mo commented Sep 23, 2025

What changes are proposed in this pull request?

Add Velox datasource add-split and read metrics in Gluten.

How was this patch tested?

UT.

@github-actions github-actions bot added the VELOX label Sep 23, 2025
@rui-mo rui-mo requested a review from FelixYBW September 23, 2025 13:45
Copy link
Contributor

@FelixYBW FelixYBW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@FelixYBW
Copy link
Contributor

data source add split time counts the time to add a split to table scan in main thread. The split may or may not be prefetched.

data source read time counts the time to read a page from parquet in table scan in main thread. The page may or maynot be prefetched. it count the decompress time as well. It doesn't count the time for lazy read. Lazy read is counted in following operators or when we do flatten.

@FelixYBW FelixYBW merged commit 0cb8b98 into apache:main Sep 23, 2025
56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants